Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove EL6 from st2chatops #153

Merged
merged 2 commits into from
Jul 3, 2020
Merged

Remove EL6 from st2chatops #153

merged 2 commits into from
Jul 3, 2020

Conversation

amanda11
Copy link
Contributor

@amanda11 amanda11 commented Jul 2, 2020

CentOS 6 / RHEL 6 is to be fully deprecated. Remove the EL6 specific code and builds. As all EL distributions support systemd, remove the st2chatops init.d service definition.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That looks good!

The following sysvinit service script is EL6-only and could be removed now:
https://github.com/StackStorm/st2chatops/blob/master/rpm/st2chatops.init

Additionally, rpm spec https://github.com/StackStorm/st2chatops/blob/master/rpm/st2chatops.spec needs a bit of cleanup. With el6 removal, all EL platforms we support are systemd now and so some logic in that spec could be simplified.

Would be great to have some better description for the PR. It's history and context that might be very helpful in the future.

@arm4b arm4b added this to the 3.3.0 milestone Jul 2, 2020
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, thanks a lot!

@arm4b arm4b merged commit bbfdb0b into StackStorm:master Jul 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants